Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add V func to access a kv value from an error #2

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

jsteenb2
Copy link
Owner

@jsteenb2 jsteenb2 commented Mar 17, 2024

was dorking around with what an accessor might look like. When we return the group of Fields, it requires a fair bit of juggling to find the important bits you're after. Addidtionally, that would create a new slice of []any that may not be warranted. With an accessor, we can avoid the []any and work with what we know the string should be, a string, and return the value of any. Additionally, the func name V is super terse, perhaps too much so. However, I kind of dig it after working with it for a bit.

@jsteenb2 jsteenb2 force-pushed the feat/field_accessor branch 2 times, most recently from 97d9a68 to 1161e21 Compare March 17, 2024 23:57
@jsteenb2 jsteenb2 force-pushed the feat/field_accessor branch from 1161e21 to 51628b8 Compare March 18, 2024 00:21
@jsteenb2 jsteenb2 merged commit 09c7f1d into main Mar 18, 2024
1 check passed
@jsteenb2 jsteenb2 deleted the feat/field_accessor branch March 18, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant